Skip to content

Add support for the esp32p4 #2934

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 2 commits into
base: floitsch/roll-esp-idf-5.4.2
Choose a base branch
from
Draft

Add support for the esp32p4 #2934

wants to merge 2 commits into from

Conversation

@floitsch
Copy link
Member

@floitsch floitsch commented Oct 19, 2025

No description provided.

@floitsch floitsch requested a review from kasperl October 19, 2025 16:20
@cla-bot cla-bot bot added the cla-signed The contributors have signed the CLA label Oct 19, 2025
@floitsch floitsch marked this pull request as draft October 19, 2025 16:21
floitsch
floitsch commented Oct 19, 2025
View reviewed changes
src/os_esp32.cc
#ifdef CONFIG_IDF_TARGET_ESP32S3
range.address = reinterpret_cast<void*>(0x3fca0000);
range.size = 384 * KB;
#elif CONFIG_IDF_TARGET_ESP32P4
Copy link
Member Author

@floitsch floitsch Oct 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need some help here. I have no idea what I'm doing...
Also, I think that the C3, C6, and S2 might need a different value too?

@floitsch
Copy link
Member Author

floitsch commented Oct 19, 2025

With the default configuration the esp32-p4 crashes immediately.
If I disable cmpct-malloc, it manages to run a bit.
However, It has a tendency to run out of memory:

Heap report @ out of memory in 'stack grow' 216:324:
  ┌───────────┬──────────┬─────────────────────────────────────────────────────┐
  │   Bytes   │  Count   │  Type                                               │
  ├───────────┼──────────┼─────────────────────────────────────────────────────┤
  └───────────┴──────────┴─────────────────────────────────────────────────────┘
  Total: 0 bytes in 0 allocations (0%), largest free 376k, total free 439k

It looks like there is memory, but Toit doesn't recognize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kasperl kasperl Awaiting requested review from kasperl

Assignees

No one assigned

Labels

cla-signed The contributors have signed the CLA

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@floitsch